Skip to content

docs: adds parameter descriptions for endpoints and incorporates the latest epidatr docs updates#45

Merged
JavierMtzRdz merged 30 commits intodevfrom
upd-doc
Feb 12, 2026
Merged

docs: adds parameter descriptions for endpoints and incorporates the latest epidatr docs updates#45
JavierMtzRdz merged 30 commits intodevfrom
upd-doc

Conversation

@JavierMtzRdz
Copy link
Contributor

Summary:

This PR provides details about the required parameters according to numpy docsting, adds references to the updated delphi-epidata documentation, and incorporates changes to the documentation from cmu-delphi/epidatr#324.

Prerequisites:

  • Unless it is a documentation hotfix it should be merged against the dev branch
  • Branch is up-to-date with the branch to be merged with, i.e. dev
  • Build is successful
  • Code is cleaned up and formatted

This helps to render NumPy style docstrings as those provided in the package.
Copy link
Contributor

@dshemetov dshemetov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't try compiling this locally, figured you already did.

praise: thanks for using the napoleon extension, definitely looks a lot cleaner.
praise: thanks for porting all the epidatr docstrings over here.

I spot checked a few and they looked good to me. We can keep tweaking in follow up PRs, but this is already an improvement.

Private API key.
locations : StringParam
Geographic locations to return. Supports a single string or a sequence of strings.
See `Geographic Codes <https://cmu-delphi.github.io/delphi-epidata/api/geographic_codes.html#us-states>`__.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do the __ at the end do?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JavierMtzRdz JavierMtzRdz merged commit bd4d722 into dev Feb 12, 2026
1 check passed
@JavierMtzRdz JavierMtzRdz deleted the upd-doc branch February 12, 2026 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants